-
Notifications
You must be signed in to change notification settings - Fork 606
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update ember-cli et al #1395
update ember-cli et al #1395
Conversation
})).to.eventually.be.rejectedWith( | ||
'Authenticators cannot extend from themself. Remove the --base-class option or specify one of "oauth2", "torii" or "devise".' | ||
); | ||
xit('throws when the authenticator is specified as its own base class', function() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These tests currently fail due to ember-cli/ember-cli#7019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
again, not a big fan of these "update everything" commits
blueprints/authenticator/index.js
Outdated
@@ -38,7 +38,7 @@ module.exports = { | |||
body: EOL + ' restore(data) {' + EOL + ' },' + EOL + EOL + ' authenticate(/*args*/) {' + EOL + ' },' + EOL + EOL + ' invalidate(data) {' + EOL + ' }' | |||
}; | |||
} else if (name === baseClass) { | |||
throw new SilentError('Authenticators cannot extend from themself. Remove the --base-class option or specify one of "oauth2", "torii" or "devise".'); | |||
throw new SilentError('Authenticators cannot extend from themselves. Remove the --base-class option or specify one of "oauth2", "torii" or "devise".'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems unrelated to the PR purpose of updating depenendencies?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, reverted
SilentError, | ||
'Authenticators cannot extend from themselves. Remove the --base-class option or specify one of "oauth2", "torii" or "devise".' | ||
); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the changed exception also seems unrelated to the PR purpose
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, was trying to fix the tests until I figured out they were breaking because of ember-cli
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reverted
package.json
Outdated
"cors": "^2.8.4", | ||
"ember-cli": "~2.14.0", | ||
"ember-cli-addon-tests": "^0.7.0", | ||
"ember-bootstrap": "^1.0.0-beta.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrong sort order
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
package.json
Outdated
"chai": "^4.1.0", | ||
"cors": "^2.8.4", | ||
"ember-cli": "~2.14.0", | ||
"ember-cli-addon-tests": "^0.7.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks like a new dependency, not an updated one? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@Turbo87: good to approve? |
🆙